Improve cli integration test#51
Merged
RohanAdwankar merged 4 commits intoRohanAdwankar:masterfrom Jan 16, 2026
Merged
Conversation
not-my-profile
force-pushed
the
cli-test
branch
from
63e51f1 to
2c9c21e
Compare
We want to be able to see which cases failed and run specific cases.
There's not much point in committing them (and having to keep them up to date would be unnecessary hassle). The rendering is done with resvg anyway.
With the default of scale=10 the png tests take 2min. With scale=1 they take 1.5s.
not-my-profile
force-pushed
the
cli-test
branch
from
2c9c21e to
1ac8edb
Compare
RohanAdwankar
approved these changes
Jan 16, 2026
Owner
|
this looks awesome and definitely will make development a smoother process 😮💨 |
Contributor
Author
|
Yeah the test speedup is nice. No worries, get well soon! Over the last week I started working on my own interactive diagram tool, though with its own syntax, similar but not the same as Mermaid, but the biggest difference being that I don't allow free positioning of nodes ... every node is on an intersection of a grid. I'll let you know if I end up releasing it so you can check it out :) |
Owner
|
yes definitely share it when you release, the grid idea sounds interesting! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If you like these changes please merge them without squashing.